Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSS override to fix table rendering in sect 7. #64

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

msdemlei
Copy link
Contributor

ivoatex usually puts rules above and below tabular-s, and that doesn't work for the pseudo tables used there.

This is a fairly lame fix that requires an extra file and, horror, an !important within the CSS.

If you consider this to be too abhorrent, I'd be ok with doing a fix within ivoatex, but for now I think the present horror is an artefact of an abuse of tables, and hence I'm reluctant to spend time on a good fix.

ivoatex usually puts rules above and below tabular-s, and that doesn't
work for the pseudo tables used there.
Copy link
Contributor

@tomdonaldson tomdonaldson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While this is a bit hacky, I think the results are enough of an improvement to be worth the hack.

I note that compared to prior versions, lines between rows are now missing in section 7 (the problem area) and in sections 2 and 2.1. I don't think this hurts the readability, so it's OK, but an example of side effects we should be aware of.

This addresses the extra lines portion of #63 but we'll leave that open as there still may be an issue (at least on my Mac) with the fonts.

@tomdonaldson tomdonaldson merged commit 76d5f2b into ivoa-std:master Dec 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants